Skip to content

Conversation

@JackyLee3362
Copy link

@JackyLee3362 JackyLee3362 commented Mar 29, 2025

I am submitting

  • a new package
  • an update for a package

A template to help ECNU students better use Typst for writing their theses, providing a structured directory in the Template and facilitating user customization.

Description: Explain what the package does and why it's useful.

I have read and followed the submission guidelines and, in particular, I

  • [✅] selected a name that isn't the most obvious or canonical name for what the package does
  • [✅] added a typst.toml file with all required keys
  • [✅] added a README.md with documentation for my package
  • [✅] have chosen a license and added a LICENSE file or linked one in my README.md
  • [✅] tested my package locally on my system and it worked
  • [✅] excluded PDFs or README images, if any, but not the LICENSE
  • [✅] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

@typst-package-check typst-package-check bot changed the title simple-ecnu-thesis:0.1.0 simple-ecnu-thesis:0.1.0 Mar 29, 2025
@typst-package-check typst-package-check bot added the new A new package submission. label Mar 29, 2025
@@ -0,0 +1,21 @@
MIT License

Copyright (c) [year] [fullname]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should update this line with actual information.


// --------------------- 具体的字体 ---------------------
// 英文字体
#let Times-New-Roman = "Times New Roman"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The most common convention for variable names in Typst package is lowercase-kebab-case. It would be nice if you could rename the few variables in this file so that they don't have capital letters.

@elegaanz
Copy link
Member

elegaanz commented May 5, 2025

Hello, just a message to remind you of the review I left a few weeks ago. If you don't have time to handle it for now, please let me know, I'll close this PR for inactivity (but then you can still submit your package again in another PR once it is ready).

@elegaanz
Copy link
Member

Closing because of inactivity, as said above, feel free to open a new PR later.

@elegaanz elegaanz closed this Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new A new package submission.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants